Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fetchgitlocal: fix permission problem #31937

Closed
wants to merge 1 commit into from
Closed

Conversation

teto
Copy link
Member

@teto teto commented Nov 22, 2017

Motivation for this change

Attempt to make fetchgitLocal usable again; see #10873 and #31363

Things done

It was not possible to change the git index within the /nix/store/ so we
copy the git folder from /nix/store/ into /tmp and operates on it via GIT_DIR.

Would be cool if someone else could try it to make sure it works. Looked ok here on my project.

  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@pvgoran
Copy link
Contributor

pvgoran commented Dec 8, 2017

I tested it (nothing complicated, just basic usage), works fine for me.

It was not possible to change the git index within the /nix/store/ so we
copy the git folder in /tmp and operates on it via GTI_DIR.
@teto
Copy link
Member Author

teto commented Feb 13, 2018

@volth sounds good but I don't find buitlins.exec in nixUnstable ?

@teto
Copy link
Member Author

teto commented Mar 8, 2018

I believe nix 2.0 fetchGit makes this obsolete. Thus closing.

@teto teto closed this Mar 8, 2018
@teto teto deleted the fetchgitlocal_pr branch May 13, 2020 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants